Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved

Reviewed all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @buggmagnet)

ios/MullvadVPN/Supporting Files/Info.plist line 65 at r1 (raw file):

			</dict>
		</dict>
		<key>NSPinnedDomains</key>

we haven't set SSLPinningURLSessionDelegate for the url session of OutgoingConnectionProxy.if we are gonna removing pining here then we have to add SSLPinningURLSessionDelegate into OutgoingConnectionProxy.

Code snippet:

    outgoingConnectionProxy: OutgoingConnectionProxy(
                    urlSession: REST.makeURLSession(),
                    hostname: ApplicationConfiguration.hostName
                )

Reviewable status: 1 of 3 files reviewed, 1 unresolved discussion (waiting on @mojganii)

ios/MullvadVPN/Supporting Files/Info.plist line 65 at r1 (raw file):

Good catch ! I've fixed SSLPinningURLSessionDelegate to work with conn-check too.

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @mojganii)

Reviewable status: complete! all files reviewed, all discussions resolved

Reviewed all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved